Skip to content

fix: include eas-abi.json in package distribution#6

Merged
cyberstorm-daemon merged 1 commit intomainfrom
fix/include-abi
Sep 9, 2025
Merged

fix: include eas-abi.json in package distribution#6
cyberstorm-daemon merged 1 commit intomainfrom
fix/include-abi

Conversation

@cyberstorm-daemon
Copy link
Member

Summary

Fixes critical packaging issue where eas-abi.json was missing from built wheel packages, causing web3.exceptions.NoABIFunctionsFound errors when creating attestations on Base Sepolia testnet.

Root Cause

The issue was caused by setuptools auto-discovery creating ambiguous package structures on case-insensitive filesystems (macOS), which prevented proper inclusion of package data files configured in pyproject.toml.

Changes Made

  • Fixed package configuration: Added explicit packages list in pyproject.toml to resolve auto-discovery conflicts
  • Enhanced package-data: Updated configuration to explicitly include JSON files in contracts directory
  • Version bump: Updated to v0.1.4
  • Regression tests: Added comprehensive test suite to prevent future packaging issues

Testing

Impact

  • Before: NoABIFunctionsFound error prevented any attestation creation
  • After: Full EAS functionality restored on Base Sepolia and all supported networks

Resolves the downstream bug report from the Cyberstorm Attestor development team.

🤖 Generated with Claude Code

- Add explicit package configuration in pyproject.toml to fix package discovery
- Configure package-data to include JSON files in contracts directory
- Bump version to 0.1.4
- Add comprehensive regression test suite with proper formatting

Fixes issue where eas-abi.json was missing from built wheel packages,
causing web3.exceptions.NoABIFunctionsFound errors on Base Sepolia testnet.

The root cause was setuptools auto-discovery creating ambiguous package
structures on case-insensitive filesystems (macOS), preventing proper
inclusion of package data files.
@cyberstorm-daemon cyberstorm-daemon merged commit 879bdf0 into main Sep 9, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant